home *** CD-ROM | disk | FTP | other *** search
/ NetNews Offline 2 / NetNews Offline Volume 2.iso / news / comp / lang / c-part2 / 14153 < prev    next >
Encoding:
Internet Message Format  |  1996-08-05  |  2.3 KB
  1. Path: erich.triumf.ca!bennett
  2. From: bennett@erich.triumf.ca (P.Bennett)
  3. Newsgroups: comp.lang.c
  4. Subject: Re: Can anyone help a newbie out ?
  5. Date: 11 Apr 1996 22:54 PST
  6. Organization: TRIUMF: Tri-University Meson Facility
  7. Distribution: world
  8. Message-ID: <11APR199622540287@erich.triumf.ca>
  9. References: <4kkf3r$5b0@darwin.nbnet.nb.ca>
  10. NNTP-Posting-Host: ftp.triumf.ca
  11. News-Software: VAX/VMS VNEWS 1.50    
  12.  
  13. In article <4kkf3r$5b0@darwin.nbnet.nb.ca>, lewwid@brunswickmicro.nb.ca (Jeff) writes...
  14. >I started reading a book on C, and i am stuck on one exercise, and i
  15. >know it's really simple, but i can't do it :(  
  16. > 
  17. >Problem:Write a function that accepts two strings.  Count the number
  18. >of characters in each string, and return the pointer to the longer
  19. >string.
  20. > 
  21. >This is what my lamer brain had to say ...
  22. > 
  23. >#include <stdio.h>
  24. > 
  25. >char *ptr, *ptr1;
  26.  
  27. This reserves space for two pointers-to-char, but does not initialize them to
  28. point anywhere in particular, not does it reserve any space to store chars. 
  29. You probably should say:
  30.     char ptr[80], ptr1[80];
  31. (or change '80' as needed to accomodate the longest expected string.)
  32.  
  33. >    puts("Please enter in the first string :");
  34. >    gets(ptr);
  35.  
  36. Two important points: 
  37.     NEVER use gets(), unless you have a Really Good Reason.
  38.     There is NEVER a good reason to use gets().
  39. Use fgets() instead. (_Always!_)
  40.     fgets(ptr, 80, stdin);
  41.  
  42. >    while (*x != NULL)
  43.  
  44. NULL is the "null pointer constant" and is not a char.  Use:
  45.     while (*x != '\0')
  46. >        total += 1
  47.                           ^ a semicolon is needed here
  48. (there is a strlen() function that will do this for you.)
  49. Also, you aren't stepping through the array.
  50. I would probably do:
  51.     int i = 0
  52.     while(x[i++] != '\0')
  53.             total += 1;
  54.  
  55. >    if (total < total2)
  56. >        return *y;
  57.  
  58. This should be "return y", to return a pointer to one array, *y returns the
  59. first char.
  60.  
  61.  
  62. Peter Bennett VE7CEI                | Vessels shall be deemed to be in sight
  63. Internet: bennett@triumf.ca         | of one another only when one can be
  64. Packet: ve7cei@ve7kit.#vanc.bc.ca   | observed visually from the other
  65. TRIUMF, Vancouver, B.C., Canada     |                          ColRegs 3(k)
  66. GPS and NMEA info and programs: ftp://sundae.triumf.ca/pub/peter/index.html
  67. or: ftp://ftp-i2.informatik.rwth-aachen.de/pub/arnd/GPS/peter/index.html
  68.  
  69.  
  70.  
  71.  
  72.  
  73.